From: Florian Müllner <fmuellner@gnome.org>
Date: Mon, 17 Oct 2011 14:00:08 +0000 (+0200)
Subject: Revert "dash: Add minor optimization to _adjustIconSize()"
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~7442
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=1721db6d8d799dc04c2edc26509ba0439416c733;p=gnome-shell.git

Revert "dash: Add minor optimization to _adjustIconSize()"

Ooops, a patch got lost in rebase ...

This reverts commit 6d95e8b98874bed67bd895a505fd198a350a69cb.
---

diff --git a/js/ui/dash.js b/js/ui/dash.js
index 36588c0d9..aa5f97270 100644
--- a/js/ui/dash.js
+++ b/js/ui/dash.js
@@ -427,20 +427,14 @@ Dash.prototype = {
 
         let firstIcon = iconChildren[0]._delegate.child._delegate.icon;
 
-        let minHeight, natHeight;
+        // Icons may be animating, so enforce the current icon size
+        // during the size request
+        let [currentWidth, currentHeight] = firstIcon.icon.get_size();
 
-        // Enforce the current icon size during the size request if
-        // the icon is animating
-        if (firstIcon._animating) {
-            let [currentWidth, currentHeight] = firstIcon.icon.get_size();
+        firstIcon.icon.set_size(this.iconSize, this.iconSize);
+        let [minHeight, natHeight] = iconChildren[0].get_preferred_height(-1);
 
-            firstIcon.icon.set_size(this.iconSize, this.iconSize);
-            [minHeight, natHeight] = iconChildren[0].get_preferred_height(-1);
-
-            firstIcon.icon.set_size(currentWidth, currentHeight);
-        } else {
-            [minHeight, natHeight] = iconChildren[0].get_preferred_height(-1);
-        }
+        firstIcon.icon.set_size(currentWidth, currentHeight);
 
 
         // Subtract icon padding and box spacing from the available height
@@ -484,15 +478,11 @@ Dash.prototype = {
             icon.icon.set_size(icon.icon.width * scale,
                                icon.icon.height * scale);
 
-            icon._animating = true;
             Tweener.addTween(icon.icon,
                              { width: targetWidth,
                                height: targetHeight,
                                time: DASH_ANIMATION_TIME,
-                               transition: 'easeOutQuad',
-                               onComplete: function() {
-                                   icon._animating = false;
-                               }
+                               transition: 'easeOutQuad'
                              });
         }
     },